Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Accept --version to show short version info, and --help for usage. #45

Merged
merged 1 commit into from
Oct 6, 2017

Conversation

Sei-Lisa
Copy link

@Sei-Lisa Sei-Lisa commented Oct 5, 2017

-V gives the long version info; --version gives a short one, easier to parse.

There was no explicit flag for showing usage, so add one that returns a graceful return code rather than a failure code.

Take the text in short_version() as "I had no better idea" and feel free to change it if desired.

@Sei-Lisa
Copy link
Author

Sei-Lisa commented Oct 5, 2017

Amended the commit text to auto-close #43 if merged.

-V gives the long version info; --version gives a short one, easier to parse.

There was no explicit flag for showing usage, so add one that returns a graceful return code rather than a failure code.

Fixes #43
@Makopo Makopo merged commit c403d75 into Makopo:master Oct 6, 2017
@Sei-Lisa Sei-Lisa deleted the sei-add-version-option branch October 6, 2017 11:03
buildersbrewery added a commit to buildersbrewery/linden-scripting-language that referenced this pull request Oct 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants